Skip to content

Conversation

@lkdvos
Copy link
Member

@lkdvos lkdvos commented Apr 17, 2025

Was slightly too late to get this in:
@mtfishman

Small change to docstring and add missing constructor.

Small change to docstring and add missing constructor.
@lkdvos lkdvos requested a review from Jutho April 17, 2025 15:53
@mtfishman
Copy link
Collaborator

Oops, thanks @lkdvos.

@Jutho
Copy link
Member

Jutho commented Apr 17, 2025

Well spotted. Should we tag a new version after merging this?

@codecov
Copy link

codecov bot commented Apr 17, 2025

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/implementations/truncation.jl 0.00% 1 Missing ⚠️
Files with missing lines Coverage Δ
src/implementations/truncation.jl 87.32% <0.00%> (-1.09%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lkdvos
Copy link
Member Author

lkdvos commented Apr 17, 2025

Yes, let's tag a new version. I think the version was already increased, so we can simply tag this commit

@lkdvos lkdvos merged commit 2f89ab5 into main Apr 17, 2025
8 of 9 checks passed
@lkdvos lkdvos deleted the docstring-fix branch April 17, 2025 17:46
Jutho pushed a commit that referenced this pull request May 11, 2025
Small change to docstring and add missing constructor.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants